Skip to content

Add "Spice as an MCP gateway" example - #1763

Merged
krinart merged 3 commits into
trunkfrom
viktor/improved-mcp
Aug 2, 2026
Merged

Add "Spice as an MCP gateway" example#1763
krinart merged 3 commits into
trunkfrom
viktor/improved-mcp

Conversation

@krinart

@krinart krinart commented May 26, 2026

Copy link
Copy Markdown
Contributor

🗣 Description

Add Spice as an MCP gateway section to MCP page with a link to mcp_server cookbook.

@github-actions

github-actions Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

✅ Pull with Spice Passed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ Has at least one of the required labels: area/blog, area/docs, area/cookbook, dependencies
  • ✅ No banned labels detected
  • ✅ Has at least one assignee: krinart, claudespice

@github-actions

Copy link
Copy Markdown
Contributor

🚀 deployed to https://5e7d61a2.spiceai-org-website.pages.dev

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the MCP feature documentation by adding an end-to-end example that positions Spice as a single MCP endpoint aggregating built-in tools plus proxied third-party MCP servers (GitHub and Jira), and links to an external cookbook for a runnable setup.

Changes:

  • Adds a new “Spice as an MCP gateway for GitHub and Jira” section with a full Spicepod configuration example.
  • Documents expected combined tool catalog contents and a Claude Code connection command.
  • Links to the mcp-server cookbook for a complete runnable example.
Comments suppressed due to low confidence (2)

website/docs/features/large-language-models/mcp.md:182

  • This sentence implies the header shown below is always required, but API key auth is optional. Consider clarifying that the header is only needed when runtime.auth.api-key is enabled.
Connect Claude Code to this gateway with a single command:

website/docs/features/large-language-models/mcp.md:185

  • The docs use X-API-Key as the canonical HTTP API key header name. Using a different casing here is inconsistent and makes copy/paste harder.
claude mcp add --transport http spice http://localhost:8090/v1/mcp --header "X-API-KEY: <key>"

Comment thread website/docs/features/large-language-models/mcp.md
@github-actions

Copy link
Copy Markdown
Contributor

🚀 deployed to https://52dd94c8.spiceai-org-website.pages.dev

@claudespice
claudespice requested a review from lukekim August 2, 2026 19:19
@claudespice

Copy link
Copy Markdown
Collaborator

This one is green and idle rather than blocked: all 7 checks pass, every review thread is resolved, and mergeStateStatus is BLOCKED only for want of a maintainer approval. No human review had ever been requested — the single request on it went to Copilot on 2026-07-27 — so it has been sitting since 2026-05-26 with nobody being asked.

Requested @lukekim, who reviewed and merged the last several spiceai/docs PRs, so the ask now lands with someone.

@krinart

krinart commented Aug 2, 2026

Copy link
Copy Markdown
Contributor Author

This PR has been validated with real github/JIRA connections. Ready to be merged

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

🚀 deployed to https://e1a8b6c4.spiceai-org-website.pages.dev

@krinart
krinart merged commit 1bf7658 into trunk Aug 2, 2026
6 checks passed
@krinart
krinart deleted the viktor/improved-mcp branch August 2, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants